-
Notifications
You must be signed in to change notification settings - Fork 135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PMM-11079 Remove Integrated Alerting #2309
Conversation
Codecov Report
@@ Coverage Diff @@
## v3 #2309 +/- ##
==========================================
+ Coverage 43.97% 44.09% +0.12%
==========================================
Files 362 351 -11
Lines 43252 41136 -2116
==========================================
- Hits 19018 18139 -879
+ Misses 22586 21434 -1152
+ Partials 1648 1563 -85
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 6 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
I believe there could be more mentions of pmm/managed/services/errors.go Line 23 in 6e252ec
I suggest we update them all 👍 |
I removed |
This PR is postponed until PMM v3 |
@BupycHuk @ademidoff I think it's ready to be merged. Tests failure is unrelated to these changes. |
PMM-11079
Link to the Feature Build: SUBMODULES-3324
If this PR adds or removes or alters one or more API endpoints, please review and add or update the relevant API documents as well:
If this PR is related to some other PRs in this or other repositories, please provide links to those PRs: